feat: sealed PolylineFormatter architecture with FormatterBuilder and PolylineOptions#228
Closed
Copilot wants to merge 22 commits intodevelop/1.0from
Closed
Conversation
…rmatterRule Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/16db7628-2cc2-4c51-982b-0264a04d7157 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/16db7628-2cc2-4c51-982b-0264a04d7157 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Copilot created this pull request from a session on behalf of
petesramek
April 7, 2026 11:41
View session
…tatic class, update base classes and options Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/66b5b819-3735-47b2-a2ec-372ae483e46b Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
…pdate PolylineOptions to two-generic form Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/d0d5745e-96e3-4147-9b10-d3faf9374824 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
…metic, doc comment, add formatter-path integration tests Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/f4994093-4128-4101-b5df-f6a031463712 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
…n messages in AbstractPolylineDecoder Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/f4994093-4128-4101-b5df-f6a031463712 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
…lue/WriteValue, GetRequiredBufferSize; use ulong for zigzag arithmetic; update block max length to 13 Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/b16b1bac-62fc-44de-9c08-88c566cf476e Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/b16b1bac-62fc-44de-9c08-88c566cf476e Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
…ine XML comments Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/14462e5e-f3ab-4b1a-aabf-b61049ff1191 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Copilot stopped work on behalf of
petesramek due to an error
April 9, 2026 11:56
Copilot stopped work on behalf of
petesramek due to an error
April 9, 2026 15:11
Replace all references to AbstractPolylineEncoder, AbstractPolylineDecoder, PolylineEncodingOptions, and PolylineEncodingOptionsBuilder with the new fluent FormatterBuilder<TCoordinate, TPolyline> API. - Features list updated to mention FormatterBuilder, PolylineFormatter, PolylineEncoder, PolylineDecoder, and PolylineOptions - Usage section rewritten with Quick Start, Building a formatter, Encoding, Decoding, and Advanced options subsections - All code examples replaced with new API patterns - FAQ updated to reference FormatterBuilder and PolylineOptions Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
… FormatterBuilder API Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/34578db3-a4b3-4d4e-b444-f9cc1f46bbc5 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/34578db3-a4b3-4d4e-b444-f9cc1f46bbc5 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
…edPolylineEncoder/Decoder for chunked encoding support Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/49b11574-fbb3-42e8-b1f2-8af7fc403268 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
Agent-Logs-Url: https://github.com/petesramek/polyline-algorithm-csharp/sessions/49b11574-fbb3-42e8-b1f2-8af7fc403268 Co-authored-by: petesramek <2333452+petesramek@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
PolylineEncodingOptions<TCoordinate>andPolylineDecodingOptions<TCoordinate>(usingHasPrevious+Previouspattern for netstandard2.1 compatibility)IChunkedPolylineEncoder<TValue, TPolyline>andIChunkedPolylineDecoder<TPolyline, TValue>interfacesEncodeoverload toPolylineEncoderimplementingIChunkedPolylineEncoderDecodeoverload toPolylineDecoderimplementingIChunkedPolylineDecoderPolylineEncoderExtensionsandPolylineDecoderExtensionsPublicAPI.Unshipped.txtwith all 20 new public API entriesHasPreviousXML documentation to clarify that formatter's baseline is always used (defaults to zero when unconfigured)